Auto merge of #1451 - alexcrichton:issue-1450, r=huonw

Examples, tests, binaries, etc, are all allowed to have hyphens (cargo does some
renaming internally).

Close #1450

Only disallow hyphens in lib target names

Examples, tests, binaries, etc, are all allowed to have hyphens (cargo does some
renaming internally).

Close #1450

Auto merge of #1443 - alexcrichton:hyphens-to-underscores, r=alexcrichton

This change allows *packages* to have hyphens in them, but they are always
translated to underscores when translated to a crate name. This means that all
crates are compiled with a `--crate-name` that has no hyphens (as well as
`--extern` directives having no hyphens).

Binaries and examples, however, are allowed to contain
hyphens in their name. The "crate name" will still have an underscore, but the
output will be in the same dasherized name.

Explicitly named targets are not allowed to have hyphens in them as well.

Normalize hyphens to underscores in crate names

This change allows *packages* to have hyphens in them, but they are always
translated to underscores when translated to a crate name. This means that all
crates are compiled with a `--crate-name` that has no hyphens (as well as
`--extern` directives having no hyphens).

Binaries, examples, benchmarks, and tests, however, are allowed to contain
hyphens in their name. The "crate name" will still have an underscore, but the
output will be in the same dasherized name.

Explicitly named targets are not allowed to have hyphens in them as well.

Auto merge of #1445 - alexcrichton:issue-727, r=alexcrichton

Closes #727

Add a test for tests in benchmarks

Closes #727

Auto merge of #1444 - alexcrichton:issue-1398, r=huonw

This commit removes the ndebug support from Cargo and also adds a new
configuration option for profiles, `debug-assertions`, which controls whether
debug assertions in the compiler are turned on or not.

Closes #1398

Remove ndebug, add config of debug assertions

This commit removes the ndebug support from Cargo and also adds a new
configuration option for profiles, `debug-assertions`, which controls whether
debug assertions in the compiler are turned on or not.

Closes #1398

Auto merge of #1441 - alexcrichton:issue-1323, r=bson

This commit is a complete overhaul of how Cargo handles compilation profiles
internally. The external interface of Cargo is not affected by this change.

Previously each Target had a Profile embedded within it. Conceptually a Target
is an entry in the manifest (a binary, benchmark, etc) and a Profile controlled
various flags (e.g. --test, -C opt-level, etc). Each Package then contained many
profiles for each possible compilation mode. For example a Package would have
one target for testing a library, benchmarking a library, documenting a library,
etc. When it came to building these targets, Cargo would filter out the targets
listed to determine what needed to be built.

This filtering was largely done based off an "environment" represented as a
string. Each mode of compilation got a separate environment string like `"test"`
or `"bench"`. Altogether, however, this approach had a number of drawbacks:

* Examples in release mode do not currently work. This is due to how examples
  are classified and how release mode is handled (e.g. the "release" environment
  where examples are meant to be built in the "test" environment).
* It is not trivial to implement `cargo test --release` today.
* It is not trivial to implement `cargo build --bin foo` where *only* the binary
  `foo` is built. The same is true for examples.
* It is not trivial to add selective building of a number of
  binaries/examples/etc.
* Filtering the list of targets to build has some pretty hokey logic that
  involves pseudo-environments like "doc-all" vs "doc". This logic is duplicated
  in a few places and in general is quite brittle.
* The TOML parser greatly affects compilation due to the time at which profiles
  are generated, which seems somewhat backwards.
* Profiles must be overridden, but only partially, at compile time becuase only
  the top-level package's profile is applied.

In general, this system just needed an overhaul. This commit made a single
change of separating `Profile` from `Target` and then basically hit `make` until
all the tests passed again. The other large architectural changes are:

* Environment strings are now entirely gone.
* Filters are implemented in a much more robust fashion.
* Release mode is now handled much more gracefully.
* The unit of compilation in the backend is no longer (package, target) but
  rather (package, target, profile). This change had to be propagated many
  location in the `cargo_rustc` module.
* The backend does not filter targets to build *at all*. All filtering now
  happens entirely in the frontend.

I'll test issues after this change lands, but the motivation behind this is to
open the door to quickly fixing a number of outstanding issues against Cargo.
This change itself is not intended to close many bugs.

Fix tests for windows

len() == 0 => is_empty()

Auto merge of #1440 - tamird:rustup, r=alexcrichton

@alexcrichton

Make sure test directory exists

Remove unneeded `allow`s

Update to rust 2015-03-22

Fix printing `Fresh` for crates

Fix trying to document build scripts

Fix documenting binaries with libraries

Fix `cargo doc`

Both tweak how deep documentation is compiled as well as fixing up the
dependencies for a documented target.

Update tests for tweaked error messages

Don't factor bins into fingerprint calculations

The dependency is just there to get them to compile, not to actually induce
recompiles.

Fix fingerprints of build scripts that aren't compiled

Update some error messages in tests

Don't compile build scripts if they aren't run

Fix `cargo run` with no binaries

Update error message for missing examples

Fix test for bins/examples

Binaries are no longer always built when examples are built, so update the test
accordingly.

List examples in binaries built

This is required for `cargo run` to work

Don't fail `cargo run` with bins + examples

If no filter is supplied, then only consider binaries as candidates, not
examples as well.

Fix a subtesting test

Libraries are no longer built if they aren't doctested, so something else needed
to be present to trigger the library being built.

Only doctest libs by default

Fix configuring lib targets with profile options

Fix cargo test --no-run

Fix test for `--test` filtering

The flag now only applies to integration tests, not all targets with the
specified name.

Remove an extraneous is_test condition

If a profile's `test` flag is set to true, then it will always generate a unit
test so the `is_test` check is redundant.

Fix cargo test filtering for binaries

This adds a new `--bin` flag to `cargo test` to specifically say that a binary
should be tested. Additionally the dependencies are tweaked such that binaries
to not depend on themselves being available.

Overhaul how cargo treats profiles

This commit is a complete overhaul of how Cargo handles compilation profiles
internally. The external interface of Cargo is not affected by this change.

Previously each Target had a Profile embedded within it. Conceptually a Target
is an entry in the manifest (a binary, benchmark, etc) and a Profile controlled
various flags (e.g. --test, -C opt-level, etc). Each Package then contained many
profiles for each possible compilation mode. For example a Package would have
one target for testing a library, benchmarking a library, documenting a library,
etc. When it came to building these targets, Cargo would filter out the targets
listed to determine what needed to be built.

This filtering was largely done based off an "environment" represented as a
string. Each mode of compilation got a separate environment string like `"test"`
or `"bench"`. Altogether, however, this approach had a number of drawbacks:

* Examples in release mode do not currently work. This is due to how examples
  are classified and how release mode is handled (e.g. the "release" environment
  where examples are meant to be built in the "test" environment).
* It is not trivial to implement `cargo test --release` today.
* It is not trivial to implement `cargo build --bin foo` where *only* the binary
  `foo` is built. The same is true for examples.
* It is not trivial to add selective building of a number of
  binaries/examples/etc.
* Filtering the list of targets to build has some pretty hokey logic that
  involves pseudo-environments like "doc-all" vs "doc". This logic is duplicated
  in a few places and in general is quite brittle.
* The TOML parser greatly affects compilation due to the time at which profiles
  are generated, which seems somewhat backwards.
* Profiles must be overridden, but only partially, at compile time becuase only
  the top-level package's profile is applied.

In general, this system just needed an overhaul. This commit made a single
change of separating `Profile` from `Target` and then basically hit `make` until
all the tests passed again. The other large architectural changes are:

* Environment strings are now entirely gone.
* Filters are implemented in a much more robust fashion.
* Release mode is now handled much more gracefully.
* The unit of compilation in the backend is no longer (package, target) but
  rather (package, target, profile). This change had to be propagated many
  location in the `cargo_rustc` module.
* The backend does not filter targets to build *at all*. All filtering now
  happens entirely in the frontend.

I'll test issues after this change lands, but the motivation behind this is to
open the door to quickly fixing a number of outstanding issues against Cargo.
This change itself is not intended to close many bugs.

Auto merge of #1433 - alexcrichton:dylib, r=huonw

This may not end up being stable for 1.0, and we don't really need much
functionality from it anyway.

Remove dependency on DynamicLibrary

This may not end up being stable for 1.0, and we don't really need much
functionality from it anyway.

Auto merge of #1431 - IvanUkhov:newlines, r=alexcrichton

Hi,

It seems that people generally prefer not having newlines after section headers in their `Cargo.toml`s. However, when one creates a new package via `cargo new`, a newline gets automatically inserted after `[package]`. Then a typical `Cargo.toml` ends up being some what inconsistent: there is a newline after `[package]`, and there are no newlines after other headers.

On top of that, the examples given on crates.io do not seem to be consistent: [sometimes](http://doc.crates.io/index.html) there is a newline after `[package]`, and [sometimes](http://doc.crates.io/manifest.html) there is none. The same applies to other section names.

I am wondering if there should be some style conventions in this regard.

Regards,
Ivan

Auto merge of #1428 - alexcrichton:update, r=alexcrichton

Make the usage of newlines in Cargo.toml consistent

Auto merge of #1429 - IvanUkhov:build-script, r=alexcrichton

Update to rust master

Fix a typo in the description of the links manifest key

Auto merge of #1425 - alexcrichton:issue-1421, r=huonw

This was accidentally broken in a recent refactoring.

Closes #1421

Re-fix the `help` command's implementation

This was accidentally broken in a recent refactoring.

Closes #1421

Auto merge of #1402 - cmr:issue-1015, r=alexcrichton

Deprecates the rustc-args metadata keyword in favor of rustc-link-lib and
rustc-link-search, which are more precise and allows for easy, correct
handling of spaces in pathnames.

Closes #1015

Add rustc_private feature to tests which need it.

Add dedicated metadata for adding -l/-L flags

Deprecates the rustc-args metadata keyword in favor of rustc-link-lib and
rustc-link-search, which are more precise and allows for easy, correct
handling of spaces in pathnames.

Closes #1015

Auto merge of #1412 - jakub-:rust-nightly, r=alexcrichton

Update to the latest nightly

Auto merge of #1409 - alexcrichton:less-compiles, r=huonw

This was previously required to ensure that all relevant metadata files would
remain in place and not get deleted, but this auto-clean behavior has since been
removed so this should no longer be necessary.

Auto merge of #1406 - alexcrichton:issue-1404, r=wycats

When loading targets to compile, be sure to use the targets from the package
that's being passed down to the compilation step, not the one that was passed in
which is overridden.

Closes #1404

Auto merge of #1410 - johnz133:patch-2, r=huonw

cargo build now outputs to /target/debug, changed doc to reflect that.
steve suggested to add description for cargo build --release
autogenerates to guide.html. Used pre and code tags instead of ``` by following previous examples.

Updated build in guide.md

autogenerates to guide.html

Don't add non-compiled packages to the graph

This was previously required to ensure that all relevant metadata files would
remain in place and not get deleted, but this auto-clean behavior has since been
removed so this should no longer be necessary.

Don't intermingle targets between packages

When loading targets to compile, be sure to use the targets from the package
that's being passed down to the compilation step, not the one that was passed in
which is overridden.

Closes #1404

Auto merge of #1396 - alexcrichton:issue-1390, r=huonw

When calculating the output directory filename, only use the "file name"
component of the target triple specified as otherwise the output could be placed
into an odd location.

Closes #1390

Improve experience with the flexible target specification

When calculating the output directory filename, only use the "file name"
component of the target triple specified as otherwise the output could be placed
into an odd location.

Closes #1390

Auto merge of #1395 - alexcrichton:profile, r=brson

* Don't use `{:?}` in profiling output, it's too noisy
* Allow an integer to be specified with `CARGO_PROFILE` indicating how many
  levels deep should be printed.

Auto merge of #1397 - alexcrichton:update, r=alexcrichton

Continuation of #1384

Fix all tests from the update to rust master

Reduce usage of unstable features

Update to rust master

Merge branch 'master' of https://github.com/Victor-Savu/cargo into update

Improve the profiling output of Cargo

* Don't use `{:?}` in profiling output, it's too noisy
* Allow an integer to be specified with `CARGO_PROFILE` indicating how many
  levels deep should be printed.

Auto merge of #1389 - jimmycuadra:patch-1, r=alexcrichton

Add documentation for the `harness` manifest option, introduced in a6589681351dc4a87b6fc268d47b7d608004b455.

Document the `harness` manifest option.

Add documentation for the `harness` manifest option, introduced in a6589681351dc4a87b6fc268d47b7d608004b455.

Reverting to rustc nightly 2015-03-04 because threadpool build fails with more recent versions.

Merge branch 'master' of https://github.com/rust-lang/cargo

Updated versions for rustc, curl, docopt and regex.

Auto merge of #1373 - alexcrichton:issue-785, r=wycats

This commit now funnels all output of Cargo by default to be in `target/debug`
instead of the bare `target` directory. This change is targeted at raising
awareness of whether a debug build is being used (as opposed to a release
build). It is also aimed at remedying a common scenario where `cargo build` is
followed by `cargo build --release` and then the debug binaries are run by
accident.

This does not yet explore the option of providing symlinks to the most recent
build, hence this commit is a breaking change due to the restructuring of the
layout of the output.

Note that this commit does **not** change the output location for documentation.
All output of `cargo doc` continues to be funneled into the `target/doc`
directory.

Closes #785

Change the default output location to target/debug

This commit now funnels all output of Cargo by default to be in `target/debug`
instead of the bare `target` directory. This change is targeted at raising
awareness of whether a debug build is being used (as opposed to a release
build). It is also aimed at remedying a common scenario where `cargo build` is
followed by `cargo build --release` and then the debug binaries are run by
accident.

This does not yet explore the option of providing symlinks to the most recent
build, hence this commit is a breaking change due to the restructuring of the
layout of the output.

Note that this commit does **not** change the output location for documentation.
All output of `cargo doc` continues to be funneled into the `target/doc`
directory.

Closes #785

Auto merge of #1387 - alexcrichton:issue-1382, r=brson

A synthetic "Package" is being created as part of this step but its paths were
pointing at the original root, not the unpacked tarball for testing. This meant
that some assumptions about the relative-ness of paths didn't quite end up
working out.

Closes #1382

Correctly set up paths when packaging

A synthetic "Package" is being created as part of this step but its paths were
pointing at the original root, not the unpacked tarball for testing. This meant
that some assumptions about the relative-ness of paths didn't quite end up
working out.

Closes #1382

Auto merge of #1365 - ches:docs, r=brson

While reading through the docs, this section broke my concentration even though the functionality's purpose is fairly obvious, because of a grammatical agreement problem and some redundant/wordy terms. I hope this is an improvement.

A couple of other minor changes:

1. "Limitations" sound like things Cargo is *incapable* of doing. The points here are conscious decisions, so calling them "restrictions" seemed more appropriate to me.
2. TOML's source is linked to on the FAQ page, but I'm pretty likely to land directly on the Configuration page from a search, so a direct link from there seemed helpful.

r? @steveklabnik for docs.

Auto merge of #1383 - alexcrichton:issue-1381, r=huonw

Ensure we don't get erroneous information about invalid log levels or such.

Closes #1381

fixed feature attribute

removed [feature(env)] attributes

Explicitly remove RUST_LOG when crawling output

Ensure we don't get erroneous information about invalid log levels or such.

Closes #1381

Updated to rust nightly 2015-03-04

Auto merge of #1376 - fenhl:patch-1, r=alexcrichton

`dependencie"` → `dependencies`

Fix a typo in Zsh completion

`dependencie"` → `dependencies`

Auto merge of #1372 - alexcrichton:flaky, r=alexcrichton

Due to random hash map traversals this test could build either crate `a` or
crate `b` first, causing the first four output lines to possibly change. Two of
the first four lines are guaranteed to be the compilation of `b` and the other
two are the compilation of `a` as well as building its build script, but no
other lines about compiling `a` can happen until the dependency `b` has finished
compiling.

Relax output restrictions to make a test less flaky

Due to random hash map traversals this test could build either crate `a` or
crate `b` first, causing the first four output lines to possibly change. Two of
the first four lines are guaranteed to be the compilation of `b` and the other
two are the compilation of `a` as well as building its build script, but no
other lines about compiling `a` can happen until the dependency `b` has finished
compiling.

Auto merge of #1370 - alexcrichton:issue-1299, r=huonw

Apparently git submodules are checked in as '/' not '\' so if a '\' leaks
through it'll end up not being found!

Closes #1299

Auto merge of #1371 - alexcrichton:issue-1289, r=huonw

Before this commit the `dep_targets` function of `Context` didn't actually
return all dependencies in the sense that unit tests (and likely examples) would
not have the same package's library listed as a dependency. This commit
rectifies the situation by ensuring that the package's library is included
whenever necessary in the dependency list.

Closes #1289

Fix dep_targets for tests depending on libraries

Before this commit the `dep_targets` function of `Context` didn't actually
return all dependencies in the sense that unit tests (and likely examples) would
not have the same package's library listed as a dependency. This commit
rectifies the situation by ensuring that the package's library is included
whenever necessary in the dependency list.

Closes #1289

Translate '\' to '/' for submodule names

Apparently git submodules are checked in as '/' not '\' so if a '\' leaks
through it'll end up not being found!

Closes #1299

Auto merge of #1368 - cyndis:lto-dyn, r=alexcrichton

When building host code, -C prefer-dynamic is passed, but that option
is mutually exclusive with -C lto. Thus when LTO is specified, rustc
errors out.

Add test for having lto enabled while having a build script

Disable LTO for host (plugin, build script) builds

When building host code, -C prefer-dynamic is passed, but that option
is mutually exclusive with -C lto. Thus when LTO is specified, rustc
errors out.

Auto merge of #1367 - alexcrichton:fix-tar, r=alexcrichton

Fixes a bug reported in alexcrichton/flate2-rs#15 where the header wasn't being
properly read.

Update tar dependency

Fixes a bug reported in alexcrichton/flate2-rs#15 where the header wasn't being
properly read.

docs: Improve grammar/wording for `cargo owner`

Auto merge of #1360 - alexcrichton:new-io, r=alexcrichton

cc @aturon, this is one giant commit.

Update to rust master and std::{io, path}

Auto merge of #1354 - mbrubeck:gh-pages-title, r=alexcrichton

This should make them easier to find via search engines, or in browser history/bookmarks.

See #1302 for an old version of this PR.

Add "Cargo" to doc titles